Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs(obstacle_avoidance_planner): update doc for obstacle avoidance #3913

Merged

Conversation

takayuki5168
Copy link
Contributor

@takayuki5168 takayuki5168 commented Jun 7, 2023

Description

Added a document for recent update of obstacle avoidance feature in the obstacle_avoidance_planner package.

Tests performed

No need

Effects on system behavior

Nothing

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@github-actions github-actions bot added type:documentation Creating or refining documentation. (auto-assigned) component:planning Route planning, decision-making, and navigation. (auto-assigned) labels Jun 7, 2023
@takayuki5168 takayuki5168 added the tag:deploy-docs Mark for deploy-docs action generation. (used-by-ci) label Jun 7, 2023
@codecov
Copy link

codecov bot commented Jun 7, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.94 ⚠️

Comparison is base (35bc65a) 15.02% compared to head (5a8b093) 14.08%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3913      +/-   ##
==========================================
- Coverage   15.02%   14.08%   -0.94%     
==========================================
  Files        1350     1449      +99     
  Lines       95916   102295    +6379     
  Branches    27295    27285      -10     
==========================================
+ Hits        14409    14411       +2     
- Misses      65719    72108    +6389     
+ Partials    15788    15776      -12     
Flag Coverage Δ *Carryforward flag
differential 40.56% <ø> (?)
total 14.08% <ø> (-0.94%) ⬇️ Carriedforward from b2f4517

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files Coverage Δ
.../rtc_manager_rviz_plugin/src/rtc_manager_panel.cpp 0.00% <ø> (ø)
...viz_plugin/src/velocity_steering_factors_panel.cpp 0.00% <ø> (ø)
...anner/include/behavior_path_planner/parameters.hpp 50.00% <ø> (ø)
...nner/include/behavior_path_planner/utils/utils.hpp 45.45% <ø> (ø)
...or_path_planner/src/behavior_path_planner_node.cpp 18.66% <ø> (+0.01%) ⬆️
.../scene_module/goal_planner/goal_planner_module.cpp 5.13% <ø> (ø)
...test/test_behavior_path_planner_node_interface.cpp 27.90% <ø> (ø)
...test_utils/src/planning_interface_test_manager.cpp 60.88% <ø> (+0.53%) ⬆️
...de_manager/src/rtc_auto_mode_manager_interface.cpp 0.00% <ø> (ø)
planning/rtc_interface/src/rtc_interface.cpp 14.39% <ø> (+0.10%) ⬆️
... and 2 more

... and 125 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
@takayuki5168 takayuki5168 force-pushed the docs/obstacle-avoidance-update branch from 9bbf5e6 to 4dc5e1c Compare June 7, 2023 12:04
@takayuki5168 takayuki5168 marked this pull request as ready for review June 7, 2023 12:06
@takayuki5168 takayuki5168 requested review from kosuke55, TakaHoribe and a team as code owners June 7, 2023 12:06
Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
@takayuki5168
Copy link
Contributor Author

@kosuke55 Could you approve the PR.

@takayuki5168 takayuki5168 merged commit 3027968 into autowarefoundation:main Jun 8, 2023
@takayuki5168 takayuki5168 deleted the docs/obstacle-avoidance-update branch June 8, 2023 03:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:planning Route planning, decision-making, and navigation. (auto-assigned) tag:deploy-docs Mark for deploy-docs action generation. (used-by-ci) type:documentation Creating or refining documentation. (auto-assigned)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants